Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(Select): valueDisplay and collapsibleItems params expanded options #3185

Merged

Conversation

RSS1102
Copy link
Contributor

@RSS1102 RSS1102 commented Nov 6, 2024

🤔 这个 PR 的性质是?

  • 日常 bug 修复
  • 新特性提交
  • 文档改进
  • 演示代码改进
  • 组件样式/交互改进
  • CI/CD 改进
  • 重构
  • 代码风格优化
  • 测试用例
  • 分支合并
  • 其他

🔗 相关 Issue

💡 需求背景和解决方案

Warning

请注意这是一次 Breaking change 会导致 valueDisplay 方法的参数 valuecollapsibleItems 方法的参数 collapsedSelectedItems 类型均发生改变。在之前的版本中这两个参数的实际数据均为 string[] 数据类型。

但是在 types 标注却分别为 SelectValue<SelectOption>T[] (其中 TT extends SelectOption = SelectOption )。

在取舍是修改其标注类型和修改输出参数选择了后者。因为在一些场景中,可能会用到用户传入的options的 option 中的许多字段进行组合或者传输。

示例见:

📝 更新日志

  • feat(Select): Select 中 valueDisplay 方法的参数 valuecollapsibleItems 方法的参数 collapsedSelectedItems 均扩充为 options

  • 本条 PR 不需要纳入 Changelog

☑️ 请求合并前的自查清单

⚠️ 请自检并全部勾选全部选项⚠️

  • 文档已补充或无须补充
  • 代码演示已提供或无须提供
  • TypeScript 定义已补充或无须补充
  • Changelog 已提供或无须提供

Copy link
Contributor

github-actions bot commented Nov 6, 2024

完成

src/select/base/Select.tsx Outdated Show resolved Hide resolved
@RSS1102 RSS1102 marked this pull request as draft November 7, 2024 01:26
@uyarn
Copy link
Collaborator

uyarn commented Nov 7, 2024

unit test得一块修改下了

@RSS1102
Copy link
Contributor Author

RSS1102 commented Nov 7, 2024

unit test得一块修改下了

get.

@RSS1102 RSS1102 marked this pull request as ready for review November 8, 2024 20:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants